Skip to content

Conversation

@iuwqyir
Copy link
Contributor

@iuwqyir iuwqyir commented Nov 12, 2024

Problem solved

https://linear.app/thirdweb/issue/BLOCK-412/add-api-key-scope

Added Nebula service scope to the service definitions and included it in the hidden services list.


PR-Codex overview

This PR introduces the nebula service scope to the project and updates related configurations and validations, enhancing the API key settings and workflow checks.

Detailed summary

  • Added nebula service scope in packages/service-utils/src/core/services.ts.
  • Updated HIDDEN_SERVICES in apps/dashboard/src/components/settings/ApiKeys/validations.ts to include nebula.
  • Modified the condition in .github/workflows/linear.yml to check for the BLOCK keyword in pull request bodies.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@changeset-bot
Copy link

changeset-bot bot commented Nov 12, 2024

🦋 Changeset detected

Latest commit: 9ae7367

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@thirdweb-dev/service-utils Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 12, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@vercel
Copy link

vercel bot commented Nov 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 11:01pm
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 11:01pm
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 11:01pm
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 11:01pm

@github-actions github-actions bot added Dashboard Involves changes to the Dashboard. packages labels Nov 12, 2024
Copy link
Contributor Author

iuwqyir commented Nov 12, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @iuwqyir and the rest of your teammates on Graphite Graphite

@codecov
Copy link

codecov bot commented Nov 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.30%. Comparing base (6771cfe) to head (040e478).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5401   +/-   ##
=======================================
  Coverage   45.30%   45.30%           
=======================================
  Files        1068     1068           
  Lines       55433    55433           
  Branches     4000     4000           
=======================================
  Hits        25112    25112           
  Misses      29634    29634           
  Partials      687      687           
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from 6771cfe
packages 40.32% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 44.15 KB (0%) 883 ms (0%) 2.3 s (-29.36% 🔽) 3.2 s
thirdweb (cjs) 104.69 KB (0%) 2.1 s (0%) 5.8 s (+8.3% 🔺) 7.9 s
thirdweb (minimal + tree-shaking) 5.72 KB (0%) 115 ms (0%) 574 ms (+414.33% 🔺) 688 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 90 ms (-56.33% 🔽) 100 ms
thirdweb/react (minimal + tree-shaking) 18.78 KB (0%) 376 ms (0%) 625 ms (+100.11% 🔺) 1.1 s

@iuwqyir iuwqyir added the merge-queue Adds the pull request to Graphite's merge queue. label Nov 12, 2024
Copy link
Contributor Author

iuwqyir commented Nov 12, 2024

Merge activity

  • Nov 12, 5:04 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 12, 5:08 PM EST: A user added this pull request to the Graphite merge queue.
  • Nov 12, 5:14 PM EST: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Vercel – thirdweb-www', 'check_linear_in_pr_body').
  • Nov 12, 5:14 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 12, 5:27 PM EST: The merge label 'merge-queue' was removed. This PR will no longer be merged by the Graphite merge queue
  • Nov 12, 5:50 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 12, 5:50 PM EST: A user added this pull request to the Graphite merge queue.
  • Nov 12, 6:02 PM EST: A user merged this pull request with the Graphite merge queue.

iuwqyir added a commit that referenced this pull request Nov 12, 2024
## Problem solved

Added Nebula service scope to the service definitions and included it in the hidden services list.
@iuwqyir iuwqyir force-pushed the 11-12-add_nebula_service_definition branch from 0e0b075 to 1865acf Compare November 12, 2024 22:09
@graphite-app graphite-app bot added merge-queue Adds the pull request to Graphite's merge queue. and removed merge-queue Adds the pull request to Graphite's merge queue. labels Nov 12, 2024
@iuwqyir iuwqyir removed the merge-queue Adds the pull request to Graphite's merge queue. label Nov 12, 2024
## Problem solved

https://linear.app/thirdweb/issue/BLOCK-412/add-api-key-scope

Added Nebula service scope to the service definitions and included it in the hidden services list.

<!-- start pr-codex -->

---

## PR-Codex overview
This PR introduces the `nebula` service scope to the project and updates the related configurations and validations accordingly. It also modifies the pull request body check to include the `BLOCK` keyword.

### Detailed summary
- Added `nebula` service scope in `packages/service-utils/src/core/services.ts` with properties: `name`, `title`, and `description`.
- Updated `HIDDEN_SERVICES` in `apps/dashboard/src/components/settings/ApiKeys/validations.ts` to include `nebula`.
- Modified the pull request body check in `.github/workflows/linear.yml` to include `BLOCK`.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dashboard Involves changes to the Dashboard. merge-queue Adds the pull request to Graphite's merge queue. packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants